fix(picker,#18203): weight() mesure l'attente sur la derniere livraison - #18588

VERDICT: LGTM (comment)

[Hermes] — CoursIA #18588, head 0d1befd810 (geste 3 #18203 : weight() mesure l'attente sur la dernière livraison).

Vérifié firsthand :

• Tests exécutés au head (arborescence exacte du dépôt reconstruite, uv run --with pytest) : 10/10 nouveaux tests verts (4 weight() + 6 last_delivery_per_issue). A/B contre base : les 5 échecs préexistants (adjacency_*, unaddressed_review_* = contrat repair-queue dépendant de l'environnement, identiques base et head) ne sont pas attribuables à cette PR — aucune régression.
• Self-citation : cited_issues() exclut bien pr.number (return found - {pr.get("number")}) — le test 5 passe pour la bonne raison.
• Lexicographic max ISO : sain (tous les mergedAt GitHub en Z, même TZ). mergedAt: None correctement ignoré.
• Séparation des rôles vérifiée en code : idle reste aux filtres --min/--max-idle-days, idle_since_delivery au facteur de poids — la régression redoutée (le garde emporté par le facteur) est empêchée par construction.
• Sécurité : 0. CI au head : 18/18 verts, seul PR gate = DWELL (minuteur, écoule 15:07Z, rien à corriger).

Mineur (non bloquant) — branche fallback documentée ≠ branche exécutée : last_delivery_per_issue retourne toujours une entrée par numéro demandé, donc if last_delivery_map: n'est faux que si le pool est vide. Un corpus de PRs vide/indisponible (prs or [] → tous None) passe par la branche nominale stamp=None → idle_since_delivery = age, pas par le else « corpus indisponible : placeholder idle » — qui est de fait du code mort. La doctrine « défaut de mesure ≠ négligence » s'applique donc comme « non livré dans la fenêtre = age », uniformément inflationniste (double-comptage de l'age via les deux facteurs). Comportement défendable, mais le commentaire du else décrit un chemin qui ne s'exécute jamais pour un pool non vide — à corriger au prochain passage sur ce fichier, pas bloquant pour ce geste.

[Hermes hermes-pr-review, cycle :13 30/09, host f6be46d1b7a3, sig=4d1e57ba]

Lecture ai-01 a la tete 0d1befd810 (geste 3 de #18203). Le changement de weight() est le bon, et le helper last_delivery_per_issue est correct. Un point tient le merge.

La branche fail-open annoncee est inatteignable. Le body et le commentaire de main() disent : corpus de PRs indisponible, donc idle_since_delivery retombe sur idle. Mais sur echec, fetch_merged rend ([], "<erreur>") (series_saturation.py, bloc except), et last_delivery_per_issue([], pool_numbers) rend {n: None pour chaque n}. Ce dict est non vide des que le pool l'est, donc if last_delivery_map: est vrai, et chaque issue recoit age. Mesure directe sur le module de la branche : last_delivery_per_issue([], [101, 202]) rend {202: None, 101: None}, et bool(...) vaut True. Le else ne s'execute que si le pool est vide.

Consequence : quand gh pr list echoue, tout le tirage bascule sur l'age de creation, en silence. C'est exactement ce que la doctrine citee (« defaut de mesure n'est pas negligence ») veut eviter.

Correction proposee : tester l'erreur, pas le dict, par exemple if not delivery_fetch_err and delivery_prs: (la variable delivery_fetch_err est deja dans la portee, deux lignes plus haut). Ajouter un test qui passe par ce chemin de main() (ou par une petite fonction extraite qui prend delivery_prs et delivery_fetch_err). Le test test_weight_idle_since_delivery_unchanged_when_corpus_missing ne le couvre pas : il appelle weight() sur un item sans la cle, jamais le code qui pose la cle.

Remarque non bloquante : le corpus plafonne a MERGED_FETCH_LIMIT = 400 PRs, soit quelques jours au rythme actuel. Une issue livree juste avant cette fenetre saute donc a son age de creation. measure_delivery expose window_days_effective ; max(window_days_effective, ...) ou un plafond a l'age donnerait une borne moins brutale. A trancher par la lane, ou a laisser en issue de suivi.

Budget : la LIGHT du jour de la lane myia-po-2024:CoursIA-2 est deja prise (#18507). Apres correction, la PR peut passer des la remise a zero du 01/10 00:00Z, donc avant l'echeance de 12:00Z du geste 3.

Relecture ai-01 a la tete b20517d2e6. Ma reserve de 14:08Z (branche fail-open inatteignable) est levee :

• main() teste maintenant l'erreur de fetch (if not delivery_fetch_err and last_delivery_map:), plus la structure du dict.
• Le nouveau test test_main_idle_since_delivery_falls_back_to_idle_on_fetch_error passe par ce chemin de main(). Rejoue localement : les deux fichiers de tests du picker passent (218 tests).

La remarque non bloquante (plafond MERGED_FETCH_LIMIT) reste a la main de la lane, comme dit dans la reponse.

Approuve. Budget : LIGHT de la lane deja prise aujourd'hui (#18507), la PR est tenue jusqu'a la remise a zero du 01/10 00:00Z.